Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

commitlog: Small tweaks #1978

Merged
merged 2 commits into from
Nov 11, 2024
Merged

commitlog: Small tweaks #1978

merged 2 commits into from
Nov 11, 2024

Conversation

kim
Copy link
Contributor

@kim kim commented Nov 11, 2024

  • Export memory Repo when feature "test" is enabled
  • Add Send + Sync + 'static constraint to Repo::Segment

Useful to allow other crates to implement tests against an in-memory
commitlog.
`Send + Sync + 'static` is implied for `Repo::Segment`. To spare
callsites from specifying the constraints, add them to the `Repo` trait.
@kim kim added the no runtime change This change does not affect the final binaries label Nov 11, 2024
@kim kim requested a review from Shubham8287 November 11, 2024 10:08
@kim kim added this pull request to the merge queue Nov 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 11, 2024
@Centril Centril added this pull request to the merge queue Nov 11, 2024
Merged via the queue into master with commit e4fcb72 Nov 11, 2024
10 checks passed
@Centril Centril deleted the kim/commitlog/tweaks branch November 11, 2024 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no runtime change This change does not affect the final binaries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants